Skip to content

docs: add Git note#13285

Merged
kgryte merged 1 commit into
developfrom
philipp/git-notes-20260705
Jul 5, 2026
Merged

docs: add Git note#13285
kgryte merged 1 commit into
developfrom
philipp/git-notes-20260705

Conversation

@Planeshifter

Copy link
Copy Markdown
Member

Description

What is the purpose of this pull request?

This pull request adds Git notes under docs/git-notes/ proposing amended commit messages for commits merged to develop in the last 24 hours whose subject lines deviate from established repository conventions. This is output from a scheduled commit-message audit routine; no source code, tests, or behavior are changed.

  • ef6a8d0: wrong scope in subject — original read feat: add C implementation for `frechet/pdf` but the change touches lib/node_modules/@stdlib/stats/base/dists/frechet/pdf/; every other "add C implementation for" commit in the repo's history (e.g. stats/base/dists/lognormal/logcdf, stats/base/dists/f/pdf, stats/base/dists/lognormal/cdf, stats/base/dists/chisquare/pdf) uses the full package-path scope. Corrected subject: feat: add C implementation for `stats/base/dists/frechet/pdf`.

All other commits from the last 24 hours were audited (subject/body spelling, PR-URL/Closes/Ref accuracy verified against GitHub, trailer well-formedness) and found correct — no note was added for them.

Related Issues

Does this pull request have any related issues?

None.

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request?

Per the audit routine's guardrails, this PR only adds new files under docs/git-notes/; it does not rewrite history on develop or touch any existing file.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

AI Assistance

When authoring the changes proposed in this PR, did you use any kind of AI assistance?

  • Yes
  • No

If you answered "yes" above, how did you use AI assistance?

  • Code generation (e.g., when writing an implementation or fixing a bug)
  • Test/benchmark generation
  • Documentation (including examples)
  • Research and understanding

Disclosure

This PR was generated by Claude Code running a scheduled commit-message audit routine: it reviewed all no-merge commits landed on develop in the last 24 hours, verified each PR-URL/Closes/Ref trailer against the actual GitHub PR/issue, and cross-checked subject-line scope conventions against prior commit history before proposing the single correction above.


@stdlib-js/reviewers


Generated by Claude Code

Signed-off-by: Claude <noreply@anthropic.com>
@kgryte kgryte marked this pull request as ready for review July 5, 2026 19:54
@kgryte kgryte requested a review from a team July 5, 2026 19:54
@stdlib-bot stdlib-bot added the Needs Review A pull request which needs code review. label Jul 5, 2026
@kgryte kgryte changed the title docs: add Git notes correcting recent commit messages docs: add Git note Jul 5, 2026
@kgryte kgryte merged commit ff3853c into develop Jul 5, 2026
32 checks passed
@kgryte kgryte deleted the philipp/git-notes-20260705 branch July 5, 2026 19:55
@kgryte kgryte removed the Needs Review A pull request which needs code review. label Jul 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants